Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Minor] Use <sup> tag for help icon in web UI page header #3659

Closed
wants to merge 1 commit into from

Conversation

JoshRosen
Copy link
Contributor

This small commit makes the (?) web UI help link into a superscript, which should address feedback that the current design makes it look like an error occurred or like information is missing.

Before:

image

After:

image

@JoshRosen
Copy link
Contributor Author

/cc @aarondav, who suggested the <sup> tag. I think this should go into branch-1.2, since that's where this feature was introduced.

@aarondav
Copy link
Contributor

The new left-hand side gradient is a bit much, but otherwise this LGTM.

@SparkQA
Copy link

SparkQA commented Dec 10, 2014

Test build #24295 has started for PR 3659 at commit bd72899.

  • This patch merges cleanly.

@JoshRosen
Copy link
Contributor Author

Yeah, I got lazy while grabbing screencaps.

@SparkQA
Copy link

SparkQA commented Dec 10, 2014

Test build #24295 has finished for PR 3659 at commit bd72899.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24295/
Test PASSed.

@JoshRosen
Copy link
Contributor Author

I'm merging this into master and branch-1.2. Thanks!

asfgit pushed a commit that referenced this pull request Dec 10, 2014
This small commit makes the `(?)` web UI help link into a superscript, which should address feedback that the current design makes it look like an error occurred or like information is missing.

Before:

![image](https://cloud.githubusercontent.com/assets/50748/5370611/a3ed0034-7fd9-11e4-870f-05bd9faad5b9.png)

After:

![image](https://cloud.githubusercontent.com/assets/50748/5370602/6c5ca8d6-7fd9-11e4-8d1a-568d71290aa7.png)

Author: Josh Rosen <joshrosen@databricks.com>

Closes #3659 from JoshRosen/webui-help-sup and squashes the following commits:

bd72899 [Josh Rosen] Use <sup> tag for help icon in web UI page header.

(cherry picked from commit f79c1cf)
Signed-off-by: Josh Rosen <joshrosen@databricks.com>
@asfgit asfgit closed this in f79c1cf Dec 10, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants